Forum PHP.pl

Witam mam skrypt up.php

[PHP] pobierz, plaintext 
mysql_connect("localhost", "root", "haslo")or die("Nie można nawiązać połączenia z bazą"); 
mysql_select_db("db")or die("Wystąpił błąd podczas wybierania bazy danych");
 
 
 
$id = $_GET['id'];
$pt = $_GET['pt'] + 1;
echo $id;
echo $pt;
 
$pytanie = "UPDATE obrazki SET punkty=".$pt." WHERE id=".$id."" ;
mysql_query($pytanie);
header("Location: index.php");
[PHP] pobierz, plaintext 

skrypt dostaje z index.php po kliknieciu na button getem dwie zmeinne, zmienna id oraz zmienna pt. do zmiennej pt dodaje jeden i zapisuje do bazy ten wynik.

odpowiada za to ta linijka w index.php

[PHP] pobierz, plaintext 
<div class="thumbs">     <li><a href="up.php?id='.$row[0].'&pt='.$row[6].'">
[PHP] pobierz, plaintext 

Jednak chcialbym to troszke upiekszyc, a najlepiej byloby, aby umiescic ten skrypt dodajacy punkta do index.php a nie do osbnego pliku i wywolywac go javascriptem po nacisnieciu buttona niestety raczkuje w tematyce javascriptu, probowalem cos tam w jquery, a czasu nie mam, aby sie od podstaw uczyc bo przed sylwestrem musze skonczyc a to wlasciwie ostatnia rzecz, ktora mi zostala do zrobienia. Prosze o pomoc. I przepraszam za polskie znaki, ale nie posiadam polskiej czcionki.

Ten post edytował daggerski 29.12.2010, 16:28:56

Użyj ajaxa, inaczej tego nie zrobisz (IMG:style_emoticons/default/winksmiley.jpg)

A czy moglbys przytoczyc jakis krotki wzor jak to zrobic bo takze z ajaxem cieniutku u mnie...

OK w ramach prezentu świątecznego prawie gotowiec z wyjaśnieniami

Funkcja ajax to instrukcja uruchamiająca AJAX z odpowiednimi parametrami u siebie wywołujesz to np tak:
Żeby ustawić parametry dla index.php to albo JS -> http://forum.php.pl/index.php?showtopic=52...rt=#entry286748
albo bezpośrednio z PHP do wyboru do koloru (IMG:style_emoticons/default/smile.gif)
To jak rozumiem -> <div class="thumbs"> <li><a href="up.php?id='.$row[0].'&pt='.$row[6].'"> ma być przesłane jako parametry więc sobie z tym pokombinuj

Plik index.php

[PHP] pobierz, plaintext 
function polacz() {
mysql_connect("localhost", "root", "haslo")or die("Nie można nawiązać połączenia z bazą"); 
mysql_select_db("db")or die("Wystąpił błąd podczas wybierania bazy danych");
 
 
 
$id = mysql_real_escape_string($_GET['id']);   //przefiltruj dane bo ktoś sprytny zrobi ci mysql injection
$pt =  mysql_real_escape_string($_GET['pt'] + 1);
echo $id;
echo $pt;
 
$pytanie = "UPDATE obrazki SET punkty=".$pt." WHERE id=".$id."" ;
mysql_query($pytanie);
}
[PHP] pobierz, plaintext 

To umieszczasz normalnie w body jako doklejenie funkcji dla inputa

[JAVASCRIPT] pobierz, plaintext 
document.getElementById('id_twojego_inputa').onclick = function() {
ajax( {
		type: "GET",
		url: "index.php.php?id=2&pt=5",		
		onError: function(msg) {
			console.warn(msg)
		},
		onSuccess: function(msg) {
			//tu jakas funkcja ktora cos robi na zwróconych danych w formie text-u lub xml ty chcesz dokonać polaczenia i aktualizacji danych w bazie więc może tak ale nie testowałem czy zadziała
                <?php polacz(); ?>
		}			
	});
}
[JAVASCRIPT] pobierz, plaintext 

To umieszczasz w sekcji head

[JAVASCRIPT] pobierz, plaintext 
function ajax( options ) {
	options = {
		type: options.type || "POST",
		url: options.url || "",
		onComplete: options.onComplete || function(){},
		onError: options.onError || function(){},
		onSuccess: options.onSuccess || function(){},
		dataType: options.dataType || "text"
	};
 
	var xml = new XMLHttpRequest();
	xml.open(options.type, options.url, true);
 
	xml.onreadystatechange = function(){
		if ( xml.readyState == 4) {
			if ( httpSuccess( xml ) ) {
				var returnData = (options.dataType=="xml")? xml.responseXML : xml.responseText
				options.onSuccess( returnData );
			} else {
				options.onError();
			}
			options.onComplete();
			xml = null;
		}
	};
 
	xml.send();
 
	function httpSuccess(r) {
		try {
			return ( r.status >= 200 && r.status < 300 || r.status == 304 || navigator.userAgent.indexOf("Safari") >= 0 && typeof r.status == "undefined")
		} catch(e) {
			return false;
		}		
	}
}
[JAVASCRIPT] pobierz, plaintext 

Trochę odbiegnę od tematu ale, @amii możesz wytłumaczyć ten kawałek kodu:

[JAVASCRIPT] pobierz, plaintext 
function httpSuccess(r) {
	try {
		return ( r.status >= 200 && r.status < 300 || r.status == 304 || navigator.userAgent.indexOf("Safari") >= 0 && typeof r.status == "undefined")
	} catch(e) {
		return false;
	}		
}
[JAVASCRIPT] pobierz, plaintext 

Kod odpowiedzi HTTP numer 206

Chyba w tym przypadku nie powinno być okej.

Jakiś wyjątek dla Safari... że niby co Safari nie zwraca poprawnych nagłówków HTTP? W ogóle w warunku najpierw porównujesz e.status a na samym końcu sprawdzasz czy jest undefined. http://frontend.pl/2010/12/undefined-jest-be/

Być może ma to jakiś głębszy sens, którego narazie nie rozumiem, wyjaśnij, rzuć linkiem bo IMHO ciekawe (IMG:style_emoticons/default/winksmiley.jpg)

#luq to co wkleiłem to nie mój kod to tak na marginesie (IMG:style_emoticons/default/winksmiley.jpg) kod pochodzi z tego notabene bardzo dobrego kursu JS -> http://www.doman.art.pl/kursjs/kurs/ajax/ajax.html

Ten kawałek kodu zwraca true lub false w zależności od tego czy została zwrócona poprawna strona kodowa. Przeglądarka może np. zwrócić kod 404 itp. co oznacza błąd, dzięki temu mamy zapewnioną obsługę wyjątku dla tego typu sytuacji.

W tym kodzie źródłowym mimo, że kurs jak napisałem jest dobry to mogą być błędy. Zauważyłem klika przerabiając kurs a kodu nie testowałem więc.... trzeba by to przetestować.

[JAVASCRIPT] pobierz, plaintext 
xml.onreadystatechange = function(){
if ( xml.readyState == 4) {    //najpierw sprawdzamy czy dane są zwrócone i gotowe do użycia. Jeśli tak readyState obiektu XMLHttpRequest (u nas xml) zwróci 4 to znaczy, że są
if ( httpSuccess( xml ) ) {   //później testujemy w wewnętrznej funkcji czy przeglądarka zwraca kody z prawidłowego przedziału. O tym Safari przeczytaj najlepiej w źródle
var returnData = (options.dataType=="xml")? xml.responseXML : xml.responseText
options.onSuccess( returnData );
} else {
options.onError();
}
options.onComplete();
xml = null;
}
};
[JAVASCRIPT] pobierz, plaintext 

Ten post edytował amii 30.12.2010, 13:42:30

Sprostowanie - nie żadnej strony kodowej a nagłówek odpowiedzi przesyłany przez serwer.

Ja dokładnie wiem co ma robić ten kod tylko zdziwił mnie ten warunek, sprawdzanie zakresu nagłówków, wyjątek dla Safari i to wszystko wrzucone w try + catch...
W podanym przez Ciebie linku dostałem w sumie wyjaśnienie. Musze obadać dokładnie nagłówki z zakresu 200 - 300 oraz 304 i zobaczyć ten błąd na Safari (IMG:style_emoticons/default/winksmiley.jpg)

Ale ja bym jednak napisał to tak:

[JAVASCRIPT] pobierz, plaintext 
function httpSuccess(r) {
  if('object' === typeof r){
    return (r.status >= 200 && r.status < 300 || 304 == r.status || navigator.userAgent.indexOf("Safari") >= 0 && 'undefined' === typeof r.status);
 
  }
  else{
    return (navigator.userAgent.indexOf("Safari") >= 0);
  }
}
[JAVASCRIPT] pobierz, plaintext 

Wtedy nie trzeba tego w try pchać (który jest w JS`ie strasznie wolny) i nie wysypie nam się z powodu braku pola status w zmiennej r.

Do tematu wracając.

Jak zapisujesz jakies dane które dodajesz o 1 to nie przekazuj ich poprzez url, bo kazdy na to wpłynie i doda więcej punktów. Przekazuj tylko id i zrób tak:

[PHP] pobierz, plaintext 
 
$pytanie = "UPDATE obrazki SET punkty=punkty+1 WHERE id=".$id."" ;
[PHP] pobierz, plaintext 

Tlumacząc:


Mam nadzieję, ze w miare ok wytłumaczyłem.